Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1434 js console errors #1435

Closed
wants to merge 59 commits into from
Closed

1434 js console errors #1435

wants to merge 59 commits into from

Conversation

kkrug
Copy link
Contributor

@kkrug kkrug commented Sep 23, 2024

Description

Add validation to prevent javascript console erros

Related Issues

GSA/challenges-and-prizes#2025

Changes Made

  • Updated index.js to add validation

Screenshots (if applicable)

N/A

Checklist

  • PR is assigned to a project: Challenge_gov Board
  • PR is assigned to a milestone: current sprint
  • GH issues are assigned to the PR (under Development section on the right hand side)
  • PR is assigned a reviewer, and requires code review and approval by a teammate
  • New functionality is tested and all tests are green
  • I have added unit tests for new functionality or updated existing tests for changes.
  • I have updated the documentation/README accordingly, if necessary.
  • If you have DB migration, it is a "safe" migration and you've confirmed it can be rolled back.
  • If applicable, Controllers modified contain appropriate authorization plugs
  • This PR has been reviewed by at least one other team member.

jdonis and others added 30 commits April 29, 2024 22:01
Updating PROD Dependencies
Revert "Updating PROD Dependencies "
Patch Quantum & html_sanitize
…ize-prod

Dependency patch quantum html sanitize prod
Jdonis/1372 serialize javascript dependabot
Adding year & new webpack configuration
kkrug and others added 27 commits August 12, 2024 13:52
Upgrade yarn in staging
@kkrug kkrug closed this Sep 23, 2024
@jdonis jdonis deleted the 1434-js-console-errors branch September 23, 2024 18:43
@r-bartlett-gsa r-bartlett-gsa added this to the Sprint 09/24/24 milestone Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants